Skip to content

[DO NOT MERGE]bump modelcontextprotocol version#13

Open
aledsage wants to merge 2 commits intodynatrace-oss:mainfrom
cloudsoft:aled/bump-modelcontextprotocol-version
Open

[DO NOT MERGE]bump modelcontextprotocol version#13
aledsage wants to merge 2 commits intodynatrace-oss:mainfrom
cloudsoft:aled/bump-modelcontextprotocol-version

Conversation

@aledsage
Copy link
Copy Markdown
Contributor

@aledsage aledsage commented Dec 15, 2025

To replace #4

WARNING: DO NOT MERGE. This change does not work correctly; it does not handle the MCP tool parameters. Initial attempts to fix this, to do it the same way as the SaaS dynatrace-mcp, have not yet worked.

Just bumping the version results in compilation failures. Need the additional code changes for it to work with the new type system. These are pretty-much the same changes as were done in the Dynatrace SaaS MCP (unfortunately fix involves using any). The code change here does go slightly beyond the Dynatrace SaaS code though, as server.tool is deprecated so we also update that to use server.registerTool instead.

Note: includes the change in #11, to test if that fix works (for github workflow to comment on the PR when package.json is being changed).

@aledsage aledsage changed the title bump modelcontextprotocol version [DO NOT MERGE]bump modelcontextprotocol version Dec 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant